Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix/#243] 선호 시간 입력 다음 버튼 활성화 안되는 문제 해결 #244

Merged
merged 2 commits into from
Jun 26, 2024

Conversation

simeunseo
Copy link
Member

✨ 해당 이슈 번호 ✨

#243

todo

  • 선호시간 입력시 다음 버튼이 정상적으로 활성화되도록 로직 변경
  • 뒤로가기 클릭시 'handle 작동' alert 삭제
  • 필요없는 console.log 정리

📌 내가 알게 된 부분

📌 공유하고 싶은 부분

선호 시간 state가 처음에 preferTimes: [{ startTime: '00:00', endTime: '00:00' }] 이렇게 초기화되어있어요.
아마 '직접 시간 입력'시에 초기값을 지정해주기 위해 이렇게 해두신 것 같다는 추측...

그런데 버튼 활성화 로직이 다음과 같이 되어있더라구요

typeState={
            meetingInfo.preferTimes.length >= 1 &&
            meetingInfo.preferTimes[0].startTime &&
            meetingInfo.preferTimes[0].endTime !== '00:00'
              ? 'primaryActive'
              : 'primaryDisabled'
          }

length가 1이상이면서, "첫번째 객체"에 startTime과 endTime이 모두 올바르게 존재할때 라는 건데
오전/오후/저녁 중에 하나를 선택하면, preferTimes에 초기값인 { startTime: '00:00', endTime: '00:00' }가 그대로 있는채로 새로운 객체가 추가돼요.
반면 직접 시간 입력을 하면 초기값이 사라지고 새로운 객체가 다시 처음으로 들어가요.

그니까 오전/오후/저녁을 선택할 경우는, meetingInfo.preferTimes[0].startTime과 endTime이 초기값으로 계속 존재하기 때문에 항상 endTime이 '00:00'이 되고, 위의 조건을 만족하지 못하게 되는 문제입니다.

preferTimes 상태를 관리하는 로직 자체를 리팩토링할 필요가 있을거같긴한데, 재훈오빠가 건드는게 나을 것 같아서
일단 hotfix를 위해서 버튼이 활성화되는 조건을 다음과 같이 고쳐서 해결했습니다.

meetingInfo.preferTimes.length >= 2 ||
      (meetingInfo.preferTimes[0].startTime !== '00:00' &&
        meetingInfo.preferTimes[0].endTime !== '00:00')

-> length가 2이상(오전/오후/저녁 중 하나가 선택됨) || preferTimes[0].startTime과 endTime이 모두 '00:00'이 아님

📌 질문할 부분

📌스크린샷

2024-06-25.5.01.03.mov

@simeunseo simeunseo requested a review from ljh0608 June 25, 2024 08:12
@simeunseo simeunseo self-assigned this Jun 25, 2024
Copy link
Member

@ljh0608 ljh0608 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR 설명도 자세하고 문제점이 무엇인지 정확히 파악했습니다! 추후 리팩토링할 때 참고해서 적용하겠습니다!

@simeunseo simeunseo merged commit 1629b27 into develop Jun 26, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants